Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Prepare release v4.12.0 #591

Merged
merged 4 commits into from
Aug 16, 2024
Merged

Prepare release v4.12.0 #591

merged 4 commits into from
Aug 16, 2024

Conversation

d-honeybadger
Copy link
Contributor

@d-honeybadger d-honeybadger commented Aug 16, 2024

Got wondering for a bit if this should be a major release or not, due to the breaking change in exernal-snapshotter we pulled in, but ultimately decided it's not a breaking change for csi-digitalocean

  • there's nothing breaking in the digitalocean/do-csi-plugin:v4.12.0 image/binary itself; users who just pull in the new image will be fine
  • if someone grabs all the crds and example deployments from our repo, they'll be fine too cause the crds and the webhook will be updated in conjunction

UPD: I'm probably overthinking it. The 4.12.0 version is for do-csi-plugin, the validating webhook configuration is just here so that we can e2e testing and for convenience

CHANGELOG.md Outdated
Update Go to 1.23
Update other go.mod dependencies
Update VolumeSnapshot, VolumeSnapshotContents, and VolumeSnapshotClasses CRDs to pull in CEL validation rules new in external-snapshotter v8 release
Remove VolumeSnapshot and VolumeSnapshotContets from the snapshot validating webhook configuration as those are now validated via CEL rules on their CRDs
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: typo with VolumeSnapshotContets

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fixed - TY!

## v4.12.0 - 2024.08.16

* Update CSI driver for Kubernetes 1.31
Update CSI sidecars to latest
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i think it will be cleaner to have them in bullet points * like done below and also link the relevant PRs 🙂

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

They're all in the same PR though

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh it's all going in the same paragraph when rendered. Let me find something better

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added newlines so that in the rendered .md those are kept on separate lines
Still a single PR though

Copy link
Contributor

@dhij dhij left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@d-honeybadger d-honeybadger merged commit 9b90ec1 into master Aug 16, 2024
8 checks passed
@d-honeybadger d-honeybadger deleted the prepare-release-v4.12.0 branch August 16, 2024 19:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants